Skip to content

Deletes features flags for shipped features#36628

Merged
martykulma merged 1 commit into
MaterializeInc:mainfrom
martykulma:maz-cleanup-sns-feature-flags
May 20, 2026
Merged

Deletes features flags for shipped features#36628
martykulma merged 1 commit into
MaterializeInc:mainfrom
martykulma:maz-cleanup-sns-feature-flags

Conversation

@martykulma
Copy link
Copy Markdown
Contributor

@martykulma martykulma commented May 19, 2026

These features are enabled globally, so removing these flags from the code.

  • enable_copy_to_expr - support for COPY TO 's3://..'
  • enable_aws_msk_iam_auth - support for IAM authentication with MSK
  • enable_copy_from_remote - support for COPY FROM <url>
  • enable_iceberg_sink - support for Iceberg table as a SINK

@martykulma martykulma requested a review from a team May 19, 2026 20:41
@martykulma martykulma force-pushed the maz-cleanup-sns-feature-flags branch from 70a68ea to 0a0cfdb Compare May 19, 2026 21:09
@martykulma martykulma changed the title feature_flag: remove enable_sql_server_source Deletes features flags for shipped features May 19, 2026
@martykulma martykulma force-pushed the maz-cleanup-sns-feature-flags branch from 0a0cfdb to 4145443 Compare May 20, 2026 09:57
@martykulma martykulma marked this pull request as ready for review May 20, 2026 10:43
@martykulma martykulma requested review from a team as code owners May 20, 2026 10:43
@martykulma martykulma force-pushed the maz-cleanup-sns-feature-flags branch from 4145443 to 064c0e7 Compare May 20, 2026 12:07
- enable_copy_to_expr,
- enable_aws_msk_iam_auth,
- enable_copy_from_remote,
- enable_iceberg_sink,
@martykulma martykulma force-pushed the maz-cleanup-sns-feature-flags branch from 064c0e7 to 02745e7 Compare May 20, 2026 13:16
@martykulma martykulma requested a review from frankmcsherry May 20, 2026 15:31
Copy link
Copy Markdown
Contributor

@patrickwwbutler patrickwwbutler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@martykulma martykulma merged commit be6145e into MaterializeInc:main May 20, 2026
343 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants